Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(meta): let stream actors in fragment share same stream node #20310

Open
wants to merge 2 commits into
base: yiming/fragment-aware-actor-graph-builder
Choose a base branch
from
Open

feat(meta): let stream actors in fragment share same stream node #20310

wants to merge 2 commits into from
+408 −249

Conversation

wenym1
Copy link
Contributor

@wenym1 wenym1 commented Jan 26, 2025

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.

What's changed and what's your intention?

Checklist

  • I have written necessary rustdoc comments.
  • I have added necessary unit tests and integration tests.
  • I have added test labels as necessary.
  • I have added fuzzing tests or opened an issue to track them.
  • My PR contains breaking changes.
  • My PR changes performance-critical code, so I will run (micro) benchmarks and present the results.
  • My PR contains critical fixes that are necessary to be merged into the latest release.

Documentation

  • My PR needs documentation updates.
Release note

@wenym1 Graphite App
Copy link
Contributor Author

wenym1 commented Jan 26, 2025 •
edited
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@wenym1 wenym1 mentioned this pull request Jan 26, 2025
Merged
8 tasks
@wenym1 wenym1 force-pushed the yiming/stream-actor-share-stream-node branch from 3a7673b to e70df67 Compare January 26, 2025 07:41
@gru-agent Gru Agent
Copy link

gru-agent bot commented Jan 26, 2025

This pull request has been modified. If you want me to regenerate unit test for any of the files related, please find the file in "Files Changed" tab and add a comment @gru-agent. (The github "Comment on this file" feature is in the upper right corner of each file in "Files Changed" tab.)

@wenym1 wenym1 force-pushed the yiming/stream-actor-share-stream-node branch 2 times, most recently from 1f3c97a to 841bcdb Compare January 26, 2025 10:37
@wenym1 wenym1 force-pushed the yiming/merge-node-no-upstream-actors branch from 16121c8 to 9946e87 Compare January 26, 2025 17:07
@wenym1 wenym1 force-pushed the yiming/stream-actor-share-stream-node branch 8 times, most recently from 383c96a to f440a4e Compare January 31, 2025 08:51
@wenym1 wenym1 force-pushed the yiming/stream-actor-share-stream-node branch from f440a4e to fea493a Compare January 31, 2025 09:13
@wenym1 wenym1 force-pushed the yiming/stream-actor-share-stream-node branch 2 times, most recently from ab5566b to 71ee2c7 Compare February 5, 2025 04:58
@wenym1 wenym1 changed the base branch from yiming/merge-node-no-upstream-actors to yiming/fragment-aware-actor-graph-builder February 5, 2025 04:58
This was referenced Feb 5, 2025
Open
Open
@wenym1 wenym1 force-pushed the yiming/fragment-aware-actor-graph-builder branch from 7bb1af8 to 8114711 Compare February 5, 2025 08:56
@wenym1 wenym1 force-pushed the yiming/stream-actor-share-stream-node branch from 71ee2c7 to b4113a5 Compare February 5, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci/main-cron/run-all ci/run-backfill-tests ci/run-e2e-single-node-tests ci/run-e2e-test-other-backends type/feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@wenym1